src/features/manager/workspace-image/ui/WorkspaceImageCarousel.tsx (1)

32-50: 🩺 Stability & Availability | 🟠 Major | ⚡ Quick win

이미지가 0개가 되면 모달이 “닫히지” 않고 전역 부작용만 남습니다.

total === 0 가드가 useEffect 뒤에 있어서, 열려 있는 동안 images가 빈 배열로 바뀌면 스크롤 락과 keydown 리스너가 유지된 채 컴포넌트만 null을 반환합니다. 지금 구조에서는 보이지 않는 모달 때문에 페이지 스크롤이 막힌 상태가 남을 수 있습니다. total === 0일 때는 전역 부작용을 걸지 말고 바로 onClose()로 닫는 쪽이 안전합니다.

수정 예시

  useEffect(() => {
+    if (total === 0) {
+      onClose()
+      return
+    }
+
     const previousOverflow = document.body.style.overflow
     document.body.style.overflow = 'hidden'

     const handleKeyDown = (event: KeyboardEvent) => {
       if (event.key === 'Escape') onClose()
       if (event.key === 'ArrowLeft') goPrev()
       if (event.key === 'ArrowRight') goNext()
     }
     window.addEventListener('keydown', handleKeyDown)

     return () => {
       document.body.style.overflow = previousOverflow
       window.removeEventListener('keydown', handleKeyDown)
     }
-  }, [onClose, goPrev, goNext])
+  }, [total, onClose, goPrev, goNext])

   if (total === 0) return null

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/features/manager/workspace-image/ui/WorkspaceImageCarousel.tsx` around
lines 32 - 50, The WorkspaceImageCarousel modal can return null when total
becomes 0, but the useEffect still applies body scroll lock and the keydown
listener, leaving global side effects behind. Move the empty-state handling so
that when total === 0 the component closes via onClose() immediately and does
not install the effect, or guard the effect itself to skip setup in that case.
Use the WorkspaceImageCarousel component, the total check, and the existing
onClose/goPrev/goNext handlers to ensure the modal fully cleans up when the
image list becomes empty.